Skip to content

[Routing] Revise routing configuration and attributes usage (routing.controllers)#21609

Merged
javiereguiluz merged 1 commit intosymfony:7.4from
NtanMidis:patch-1
Feb 12, 2026
Merged

[Routing] Revise routing configuration and attributes usage (routing.controllers)#21609
javiereguiluz merged 1 commit intosymfony:7.4from
NtanMidis:patch-1

Conversation

@NtanMidis
Copy link
Contributor

@NtanMidis NtanMidis commented Nov 29, 2025

Updated routing configuration details and corrected file paths.

Q A
Branch? 7.4
Bug fix? no
New feature? no
Deprecations? no
Issues #21558
License MIT
Updated routing configuration details

controllers:
resource: routing.controllers

Issue : #21558

SymfonyCon Amsterdam 2025 Hackathon

Updated routing configuration details and corrected file paths.|Q	|A|
|Branch?	|7.4|
|Bug fix?	|no|
|New feature?	|no|
|Deprecations?	|no|
I|ssues	|-|
|License	|MIT|

Updated routing configuration details 

controllers:
    resource: routing.controllers

Issue : symfony#21558
@carsonbot carsonbot added this to the 7.4 milestone Nov 29, 2025
@carsonbot carsonbot changed the title Revise routing configuration and attributes usage (routing.controllers) Revise routing configuration and attributes usage (routing.controllers) Nov 29, 2025
@Mynyx
Copy link
Contributor

Mynyx commented Feb 6, 2026

Also the configuration examples in the sections "Localized Routes (i18n)" and "Forcing HTTPS on Generated URLs" should be amended.

@javiereguiluz javiereguiluz merged commit f5dd32e into symfony:7.4 Feb 12, 2026
@javiereguiluz
Copy link
Member

Merged! Thanks Andreas.

@Mynyx yes, we fixed a few other occurrences of config/routes/attributes.yaml while merging. Thanks.

@carsonbot carsonbot changed the title Revise routing configuration and attributes usage (routing.controllers) [Routing] Revise routing configuration and attributes usage (routing.controllers) Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants